CI: Run lint checks once - #3935
Conversation
Avoid repeating version-independent lockfile and lint checks across the Python matrix. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
🟢 Approval recommended
No unresolved issues were identified, and coverage remains intact across supported Python versions.
Pull request overview
This PR reduces redundant CI work by running lockfile validation and linting only on Python 3.12 while preserving tests and coverage across the matrix.
Changes:
- Restricts
uv lock --checkandmake lintto Python 3.12. - Retains installation, testing, and coverage for all supported versions.
File summaries
| File | Summary |
|---|---|
.github/workflows/python-ci.yml |
Adjusts matrix conditions for lockfile checks and linting. |
Review details
- Files reviewed: 1/1 changed files
- Comments generated: 0
- Review effort level: Lite
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
geruh
left a comment
geruh
left a comment
There was a problem hiding this comment.
shouldn't we target the lowest supported version of python? I vaguely remember a lot of changes when deprecating 3.9. Especially since there have also been instances where lint passed for my 3.12 but failed for other versions.
Keep linting deduplicated while checking syntax and typing against the oldest supported interpreter.\n\nCo-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
yea thats a good idea, changed to 3.10 |
Run the lockfile check and linters only on Python 3.12 instead of repeating them across the full matrix.
The matrix still installs, tests, and reports coverage for every supported Python version.